name: pre-commit run_id: commands[2] env HOME: /home/jenkins env LANG: C.UTF-8 env PATH: /w/workspace/transportpce-tox-verify-transportpce-master/.tox/pre-commit/bin:/opt/pyenv/bin:/tmp/venv-IXLU/bin:/opt/pyenv/shims:/home/jenkins/.local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin:/opt/puppetlabs/bin env PYTHONHASHSEED: 1299773017 env PIP_DISABLE_PIP_VERSION_CHECK: 1 env PYTHONIOENCODING: utf-8 env OLM_TIMER1: 3000 env OLM_TIMER2: 2000 env TOX_ENV_NAME: pre-commit env TOX_WORK_DIR: /w/workspace/transportpce-tox-verify-transportpce-master/.tox env TOX_ENV_DIR: /w/workspace/transportpce-tox-verify-transportpce-master/.tox/pre-commit env VIRTUAL_ENV: /w/workspace/transportpce-tox-verify-transportpce-master/.tox/pre-commit metadata pid: 29891 cwd: /w/workspace/transportpce-tox-verify-transportpce-master/tests allow: /w/workspace/transportpce-tox-verify-transportpce-master/.tox/pre-commit/bin/*:sh:cpan:sudo:which:fixCIcentOS8reposMirrors.sh cmd: pre-commit run --all-files --show-diff-on-failure exit_code: 1 [WARNING] hook id `remove-tabs` uses deprecated stage names (commit) which will be removed in a future version. run: `pre-commit migrate-config` to automatically fix this. [WARNING] hook id `perltidy` uses deprecated stage names (commit) which will be removed in a future version. run: `pre-commit migrate-config` to automatically fix this. [INFO] Initializing environment for https://github.com/pre-commit/pre-commit-hooks. [WARNING] repo `https://github.com/pre-commit/pre-commit-hooks` uses deprecated stage names (commit, push) which will be removed in a future version. Hint: often `pre-commit autoupdate --repo https://github.com/pre-commit/pre-commit-hooks` will fix this. if it does not -- consider reporting an issue to that repo. [INFO] Initializing environment for https://github.com/jorisroovers/gitlint. [INFO] Initializing environment for https://github.com/jorisroovers/gitlint:./gitlint-core[trusted-deps]. [INFO] Initializing environment for https://github.com/Lucas-C/pre-commit-hooks. [INFO] Initializing environment for https://github.com/pre-commit/mirrors-autopep8. [INFO] Initializing environment for https://github.com/perltidy/perltidy. [INFO] Installing environment for https://github.com/pre-commit/pre-commit-hooks. [INFO] Once installed this environment will be reused. [INFO] This may take a few minutes... [INFO] Installing environment for https://github.com/Lucas-C/pre-commit-hooks. [INFO] Once installed this environment will be reused. [INFO] This may take a few minutes... [INFO] Installing environment for https://github.com/pre-commit/mirrors-autopep8. [INFO] Once installed this environment will be reused. [INFO] This may take a few minutes... [INFO] Installing environment for https://github.com/perltidy/perltidy. [INFO] Once installed this environment will be reused. [INFO] This may take a few minutes... trim trailing whitespace.................................................Failed - hook id: trailing-whitespace - exit code: 1 - files were modified by this hook Fixing tests/transportpce_tests/network/test01_topo_extension.py Tabs remover.............................................................Passed autopep8.................................................................Failed - hook id: autopep8 - files were modified by this hook perltidy.................................................................Passed pre-commit hook(s) made changes. If you are seeing this message in CI, reproduce locally with: `pre-commit run --all-files`. To run `pre-commit` as part of git workflow, use `pre-commit install`. All changes made by hooks: diff --git a/tests/transportpce_tests/network/test01_topo_extension.py b/tests/transportpce_tests/network/test01_topo_extension.py index 2294eacf..7ed5bf2d 100644 --- a/tests/transportpce_tests/network/test01_topo_extension.py +++ b/tests/transportpce_tests/network/test01_topo_extension.py @@ -124,7 +124,7 @@ class TransportPCEtesting(unittest.TestCase): 'ROADM-TC1-ROADM-TC1-SRG1-PP1-TXRXtoSPDR-SC1-XPDR1-XPDR1-NETWORK1', 'transportpce-or-network-augmentation:link-class': 'alien-to-tapi' } - + # uuid_A = uuid.UUID(bytes('SPDR-SA1-XPDR1+DSR+eOTSI+XPDR1-NETWORK1', 'utf-8')) # uuid_C = uuid.UUID(bytes('SPDR-SC1-XPDR1+DSR+eOTSI+XPDR1-NETWORK1', 'utf-8')) @@ -198,7 +198,7 @@ class TransportPCEtesting(unittest.TestCase): response = test_utils.transportpce_api_rpc_request( 'transportpce-networkutils', 'init-xpdr-rdm-links', {'links-input': {'xpdr-node': 'SPDR-SA1', 'xpdr-num': '1', 'network-num': '1', - 'rdm-node': 'ROADM-TA1', 'termination-point-num' : 'SRG1-PP1-TXRX', + 'rdm-node': 'ROADM-TA1', 'termination-point-num': 'SRG1-PP1-TXRX', 'rdm-topology-uuid': 'a21e4756-4d70-3d40-95b6-f7f630b4a13b', 'rdm-nep-uuid': '3c3c3679-ccd7-3343-9f36-bdb7bea11a84', 'rdm-node-uuid': 'f929e2dc-3c08-32c3-985f-c126023efc43'}}) @@ -210,7 +210,7 @@ class TransportPCEtesting(unittest.TestCase): response = test_utils.transportpce_api_rpc_request( 'transportpce-networkutils', 'init-rdm-xpdr-links', {'links-input': {'xpdr-node': 'SPDR-SA1', 'xpdr-num': '1', 'network-num': '1', - 'rdm-node': 'ROADM-TA1', 'termination-point-num' : 'SRG1-PP1-TXRX', + 'rdm-node': 'ROADM-TA1', 'termination-point-num': 'SRG1-PP1-TXRX', 'rdm-topology-uuid': 'a21e4756-4d70-3d40-95b6-f7f630b4a13b', 'rdm-nep-uuid': '3c3c3679-ccd7-3343-9f36-bdb7bea11a84', 'rdm-node-uuid': 'f929e2dc-3c08-32c3-985f-c126023efc43'}}) @@ -222,7 +222,7 @@ class TransportPCEtesting(unittest.TestCase): response = test_utils.transportpce_api_rpc_request( 'transportpce-networkutils', 'init-xpdr-rdm-links', {'links-input': {'xpdr-node': 'SPDR-SC1', 'xpdr-num': '1', 'network-num': '1', - 'rdm-node': 'ROADM-TC1', 'termination-point-num' : 'SRG1-PP1-TXRX', + 'rdm-node': 'ROADM-TC1', 'termination-point-num': 'SRG1-PP1-TXRX', 'rdm-topology-uuid': 'a21e4756-4d70-3d40-95b6-f7f630b4a13b', 'rdm-nep-uuid': 'e5a9d17d-40cd-3733-b736-cc787a876195', 'rdm-node-uuid': '7a44ea23-90d1-357d-8754-6e88d404b670'}}) @@ -234,7 +234,7 @@ class TransportPCEtesting(unittest.TestCase): response = test_utils.transportpce_api_rpc_request( 'transportpce-networkutils', 'init-rdm-xpdr-links', {'links-input': {'xpdr-node': 'SPDR-SC1', 'xpdr-num': '1', 'network-num': '1', - 'rdm-node': 'ROADM-TC1', 'termination-point-num' : 'SRG1-PP1-TXRX', + 'rdm-node': 'ROADM-TC1', 'termination-point-num': 'SRG1-PP1-TXRX', 'rdm-topology-uuid': 'a21e4756-4d70-3d40-95b6-f7f630b4a13b', 'rdm-nep-uuid': 'e5a9d17d-40cd-3733-b736-cc787a876195', 'rdm-node-uuid': '7a44ea23-90d1-357d-8754-6e88d404b670'}}) @@ -262,9 +262,9 @@ class TransportPCEtesting(unittest.TestCase): 'transportpce-networkutils', 'init-inter-domain-links', {'a-end': {'rdm-node': 'ROADM-A1', 'deg-num': '1', 'termination-point': 'DEG1-TTP-TXRX'}, 'z-end': {'rdm-node': 'ROADM-TA1', 'deg-num': '2', 'termination-point': 'DEG2-TTP-TXRX', - 'rdm-topology-uuid': 'a21e4756-4d70-3d40-95b6-f7f630b4a13b', - 'rdm-nep-uuid': 'd42ed13c-d81f-3136-a7d8-b283681031d4', - 'rdm-node-uuid': 'f929e2dc-3c08-32c3-985f-c126023efc43'}}) + 'rdm-topology-uuid': 'a21e4756-4d70-3d40-95b6-f7f630b4a13b', + 'rdm-nep-uuid': 'd42ed13c-d81f-3136-a7d8-b283681031d4', + 'rdm-node-uuid': 'f929e2dc-3c08-32c3-985f-c126023efc43'}}) self.assertEqual(response['status_code'], requests.codes.ok) print(response['output']['result']) time.sleep(2) @@ -274,9 +274,9 @@ class TransportPCEtesting(unittest.TestCase): 'transportpce-networkutils', 'init-inter-domain-links', {'a-end': {'rdm-node': 'ROADM-C1', 'deg-num': '2', 'termination-point': 'DEG2-TTP-TXRX'}, 'z-end': {'rdm-node': 'ROADM-TC1', 'deg-num': '1', 'termination-point': 'DEG1-TTP-TXRX', - 'rdm-topology-uuid': 'a21e4756-4d70-3d40-95b6-f7f630b4a13b', - 'rdm-nep-uuid': 'fb3a00c1-342f-3cdc-b83d-2c257de298c1', - 'rdm-node-uuid': '7a44ea23-90d1-357d-8754-6e88d404b670'}}) + 'rdm-topology-uuid': 'a21e4756-4d70-3d40-95b6-f7f630b4a13b', + 'rdm-nep-uuid': 'fb3a00c1-342f-3cdc-b83d-2c257de298c1', + 'rdm-node-uuid': '7a44ea23-90d1-357d-8754-6e88d404b670'}}) self.assertEqual(response['status_code'], requests.codes.ok) print(response['output']['result']) response = test_utils.get_ietf_network_request('openroadm-topology', 'config') @@ -304,15 +304,15 @@ class TransportPCEtesting(unittest.TestCase): linkType = link['org-openroadm-common-network:link-type'] if 'transportpce-or-network-augmentation:link-class' in link.keys(): linkClass = link['transportpce-or-network-augmentation:link-class'] - if (linkType == 'ROADM-TO-ROADM' and linkClass == 'inter-domain') : + if (linkType == 'ROADM-TO-ROADM' and linkClass == 'inter-domain'): find = linkId in check_list self.assertEqual(find, True) interDomainLinkNber += 1 - if (linkType == 'XPONDER-OUTPUT' and linkClass == 'alien-to-tapi') : + if (linkType == 'XPONDER-OUTPUT' and linkClass == 'alien-to-tapi'): find = linkId in check_list self.assertEqual(find, True) alienToTapiLinkNber += 1 - if (linkType == 'XPONDER-INPUT' and linkClass == 'alien-to-tapi') : + if (linkType == 'XPONDER-INPUT' and linkClass == 'alien-to-tapi'): find = linkId in check_list self.assertEqual(find, True) alienToTapiLinkNber += 1